Skip to content

Manchester | 26-ITP-Sep | Mathew Shaibu | Sprint 2 | Module data group sprint 2 - #1499

Open
MattCarolus10 wants to merge 43 commits into
CodeYourFuture:mainfrom
MattCarolus10:Module-data-group-sprint-2
Open

Manchester | 26-ITP-Sep | Mathew Shaibu | Sprint 2 | Module data group sprint 2#1499
MattCarolus10 wants to merge 43 commits into
CodeYourFuture:mainfrom
MattCarolus10:Module-data-group-sprint-2

Conversation

@MattCarolus10

@MattCarolus10 MattCarolus10 commented Nov 6, 2026

Copy link
Copy Markdown

Self checklist

  • I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
  • My changes meet the requirements of the task
  • I have tested my changes
  • My changes follow the style guide

Task code

CYF-1083

Changelist

I completed debugging task, implemented test and created function and interpreted codes

@MattCarolus10
object prep
8531191
@MattCarolus10
fix houseNumber error
037311d
@MattCarolus10
fix author object iteration
13f83e1
@MattCarolus10
add tests for parseQueryString function to validate query string parsing
3d72ba1
@MattCarolus10
add function to double all numbers in an array
454659b
@MattCarolus10
fix formatting in address.js comments for clarity
af702ab
@MattCarolus10
fix recipe logging to display ingredients correctly
e6ed80a
@MattCarolus10
fix comments in recipe.js to clarify logging functionality
e80fd6f
@MattCarolus10
implement test for contains function to check empty object case
74c65b7
@MattCarolus10
implement contains function to check if property exists in object
c61f58e
@MattCarolus10
refactor contains.test.js to improve test structure and formatting
438293b
@MattCarolus10
add test for contains function to verify behavior with missing property
4e217bc
@MattCarolus10
add test to ensure contains function throws error for array input
b72528b
@MattCarolus10
refactor contains function to validate input type before checking pro…
e8cf923 
…perty existence
@MattCarolus10
remove redundant comments and acceptance criteria from contains.test.js
8e89f1d
@MattCarolus10
implement test for createLookup function to validate country-currency…
c39280d 
… pairs
@MattCarolus10
fix: update test to validate correct output for createLookup function
80034ab
@MattCarolus10
implement createLookup function to map country-currency pairs
8c1878d
@MattCarolus10
refactor: improve formatting of createLookup test and remove redundan…
62d2398 
…t comments
@MattCarolus10
refactor: simplify parseQueryString function by removing redundant ch…
b1fdc4d 
…ecks and comments
@MattCarolus10
refactor: clean up parseQueryString function by removing unnecessary …
05ccd8c 
…comments and whitespace
@MattCarolus10
fix: correct value assignment in parseQueryString and decode URI comp…
fe222e0 
…onents
@MattCarolus10
fix: decode URI components in parseQueryString function
89d8fda
@MattCarolus10
fix: decode URI components in parseQueryString function
17adfe9
@MattCarolus10
refactor: improve key and value decoding in parseQueryString function
4918e85
@MattCarolus10
test: add test for handling empty query string
f573273
@MattCarolus10
test: add test for handling values with both + and percent-encoding
05ee028
@MattCarolus10
test: implement test for tally function with an empty array
15fe829
@MattCarolus10
test: fix indentation in tally test cases
8eb4f2d
@MattCarolus10
test: add assertion for counting unique items in tally function
967cdf6
@MattCarolus10
test: add tests for handling invalid input in tally function
1492450
@MattCarolus10
refactor: remove unnecessary comments and documentation from tally te…
d2fb260 
…st file
@MattCarolus10
feat: implement tally function to count occurrences in an array
77dc8de
@MattCarolus10
fix: correct condition to check if input is an array in tally function
5cb05ee
@MattCarolus10
fix: correct implementation of invert function to properly swap keys …
ef80285 
…and values
@MattCarolus10
fix: update module export for invert function
d2795ff
@MattCarolus10
fix: update comments in invert function to clarify expected behavior …
6eff4d9 
…and return values
@MattCarolus10
test: add unit test for invert function to verify key-value swapping
ad4efb5
@MattCarolus10
fix: add missing newline and correct syntax in invert test cases
e18c645
@MattCarolus10
fix: correct formatting and add missing newline in invert test cases
3deb65c
@MattCarolus10
test: add unit test for handling empty object in invert function
8ad6494
@MattCarolus10
fix: correct expected output for key-value swapping test case
4476d4a
@MattCarolus10
Remove prep files from sprint branch
5ab6b3e
@MattCarolus10 MattCarolus10 added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Nov 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@MattCarolus10